Skip to content

MINOR: [Release] Fix loop in universal2 wheel verification#14479

Merged
kou merged 2 commits into
apache:masterfrom
lidavidm:lidavidm-patch-2
Oct 22, 2022
Merged

MINOR: [Release] Fix loop in universal2 wheel verification#14479
kou merged 2 commits into
apache:masterfrom
lidavidm:lidavidm-patch-2

Conversation

@lidavidm

@lidavidm lidavidm commented Oct 22, 2022

Copy link
Copy Markdown
Member

No description provided.

kou
kou approved these changes Oct 22, 2022
View reviewed changes

@kou kou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit 33f2c0e into apache:master Oct 22, 2022
@ursabot

ursabot commented Oct 27, 2022

Copy link
Copy Markdown

Benchmark runs are scheduled for baseline = 9471587 and contender = 33f2c0e. 33f2c0e is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Failed] ec2-t3-xlarge-us-east-2
[Failed] test-mac-arm
[Finished ⬇️0.0% ⬆️0.0%] ursa-i9-9960x
[Failed] ursa-thinkcentre-m75q
Buildkite builds:
[Failed] 33f2c0ec ec2-t3-xlarge-us-east-2
[Failed] 33f2c0ec test-mac-arm
[Finished] 33f2c0ec ursa-i9-9960x
[Failed] 33f2c0ec ursa-thinkcentre-m75q
[Failed] 94715878 ec2-t3-xlarge-us-east-2
[Failed] 94715878 test-mac-arm
[Finished] 94715878 ursa-i9-9960x
[Failed] 94715878 ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

kou pushed a commit that referenced this pull request Nov 15, 2022
@lidavidm @kou
MINOR: [Release] Fix loop in universal2 wheel verification (#14479)
d05dd54 
Authored-by: David Li <li.davidm96@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kou kou kou approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lidavidm @ursabot @kou